Skip to content

Build perf Phase 1: bound CARGO_BUILD_JOBS, move provenance off GitHub-hosted#17

Open
noahgift wants to merge 1 commit intomainfrom
feat/build-perf-phase1-yaml
Open

Build perf Phase 1: bound CARGO_BUILD_JOBS, move provenance off GitHub-hosted#17
noahgift wants to merge 1 commit intomainfrom
feat/build-perf-phase1-yaml

Conversation

@noahgift
Copy link
Copy Markdown
Contributor

Summary

Build-performance spec Phase 1 (YAML-only, low-risk). Spec: infra/docs/specifications/build-performance.md.

  • §7.1 Phase 1 — add CARGO_BUILD_JOBS: "4" default to top-level env: so cargo doesn't oversubscribe when many runners share a host. Derived from §4.1: runners × CARGO_BUILD_JOBS ≤ 1.2 × ncores.
  • §3.8 — migrate provenance job from ubuntu-latest to [self-hosted, clean-room]. No GitHub-hosted runners in the sovereign stack.

Out of scope (follow-up PRs)

  • Lint/test/build label splitting (§7.2)
  • nextest migration (Phase 2)
  • New jobs / runner-class reshuffle

Test plan

  • gate check passes on this PR
  • Downstream repos still green after merge (sovereign-ci.yml is consumed by ~38 repos)
  • Provenance job schedules onto a clean-room-labeled runner

…fs paiml/infra#PMAT-156)

Build-performance spec Phase 1 (YAML-only, safe changes):

- Add CARGO_BUILD_JOBS=4 default to top-level env. Prevents cargo
  oversubscription when many runners share a host.
  Spec: build-performance.md §4.1 (runners × jobs ≤ 1.2 × ncores).

- Migrate provenance job from ubuntu-latest to [self-hosted, clean-room].
  Spec principle §3.8: no GitHub-hosted runners.

Follow-ups (not in this PR): lint/test/build label splitting (§7.2),
nextest migration (Phase 2).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant